Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Online image change: changes for new admintools options in 11.1.0 #142

Merged
merged 5 commits into from
Jan 19, 2022

Conversation

spilchen
Copy link
Collaborator

  • when running AT start_db, we need to run from one of the primary nodes. It won't work if we try from a read-only node that isn't being restarted.
  • when calling re_ip, use the --force option. This option is new in 11.1.0, so we needed conditional logic to know when we could use this.
  • when calling start_db, we use the host list. This option exists first in 11.0.1, so like reip, we needed conditional logic to know when we can use that option

One thing that isn't related to the title of this PR is some new logic needed in DBAddNodeReconicler. That reconciler will now requeue if some pods aren't yet ready. This was needed so that the upgrade properly waits for the transient subcluster to scale out. Prior to this change, it was possible that the image change went ahead and restarted the primaries before the transient was up. This should be solved now.

Matt Spilchen added 3 commits January 17, 2022 15:40
- use host list with start_db
- use --force option with re_ip
- initiate start_db from an offline node.  It fails if you use one of the
 read-only pods
@spilchen spilchen self-assigned this Jan 18, 2022
Copy link
Collaborator

@roypaulin roypaulin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good

@spilchen spilchen merged commit fc5bc85 into vertica:online-upgrade Jan 19, 2022
@spilchen spilchen deleted the online-upgrade-restart branch January 19, 2022 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants